Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove unused release workflow #732

Closed
wants to merge 137 commits into from
Closed

Remove unused release workflow #732

wants to merge 137 commits into from

Conversation

jankapunkt
Copy link

This PR removes the already unused release-workflow, since we for now do it manually until there is concensus and a working workflow that automates the release process.

Note: this does not remove or affect the workflows for unit tests and tests for release candidates.

HappyZombies and others added 30 commits October 8, 2021 07:48
@HappyZombies
Version 4 with readme, package.json and travis.yml updates
3c9d7a6
@HappyZombies
Remove badges for now
ba02a31
@HappyZombies
update npm install commands in readme
aba6a3e
@HappyZombies
Merge pull request #1 from node-oauth/feature.4.0.0
6da8984 
v4.0.0 initial release
@HappyZombies
Ran npm init with scope, new properties added to package.json
a0a6fdb
@HappyZombies
Merge pull request #3 from node-oauth/feature.new-org
83b3283 
feature: update package.json name
@jankapunkt
Create SECURITY.md
b9aae53
@jankapunkt
added test coverage using nyc
17e2729
@jankapunkt
Create tests.yml
a88bb4d
@jankapunkt
uncomment line for coverage test until cov is added
917b4b9
@jankapunkt
ci change push branch dev to development
dcd8a3d
@HappyZombies
Merge pull request #14 from node-oauth/add-ci
cf1ab06 
Add CI
@HappyZombies
Merge pull request #13 from node-oauth/test-coverage
b7b8af6 
added test coverage using nyc
@jankapunkt
docs: added contribution guide draft
4a9eb7b
@HappyZombies
Remove statuses, use http.STATUS_CODES
534b5e7
@jankapunkt
updated development guidelines
1722039
@jankapunkt
contribution guidelines added guiding principles
6a0c93c
@jankapunkt
pull request template added
c833a37
@jankapunkt
docs removed maintainers section from contribution guide
de41dc7
@HappyZombies
Merge pull request #26 from node-oauth/feature.http-status
4d893e4 
Remove statuses package, use built in http.STATUS_CODES
@jwerre
removed jshint added eslint
e650264
@jwerre
merge with development branch
9ebf817
@jwerre
removed should added chai
18d2404
@jwerre
removed some log statements
05b8944
@jwerre
removed 'no-prototype-builtins' from eslint
0cf3b11
@oklas
docs: add commit message convention and coding rules
ca47721
@jankapunkt
tests validator is tested for all unicode ranges
b4e344d
@jankapunkt
validator is add missing variable in is.uchar
f57f4c3
@jankapunkt
tests validator also test for multiple characters
615bd75
@jankapunkt
eslint enable no-control-regex and disable only in validator
c0e4ef1
@jankapunkt
feature(ci): release workflow only runs on non-draft pull requests
13aacce
@jankapunkt
fix(ci): use actions v2 for setting up node
edd8160
@jankapunkt
fix(ci): use code-ql only on non-draft releases
69a3043
@jankapunkt
fix(ci): fix audit job syntax error
36e13eb
@jankapunkt
fix(ci): fix publish jobs syntax error
4188547
@jankapunkt
fix(ci): fix publish jobs syntax error
ecb072e
@jankapunkt
fix(ci): fix check coverage
510546b
@jankapunkt
fix(ci): run workflows also only on push for non-draft PR
88a7bfe
@jankapunkt
fix(ci): fix tets.yml semantic error
d6ec7d8
@jankapunkt
fix(ci): release-workflow trigger on push to PR
e556def
@jankapunkt
fix(ci): add lcov reporter to test:coverage
465b786
@jankapunkt
fix(ci): use oauthjs/express-oauth-server for integration testing
b7867e7
@jankapunkt
fix(ci): fix path for integration test
3167a8e
@jankapunkt
fix(ci): fix path for integration test
c99001c
@jankapunkt
fix(ci): fix path for integration test
fd9ead8
@jankapunkt
fix(ci): fix path for integration test
aee6d92
@jankapunkt
fix(ci): fix path for integration test
916c613
@jankapunkt
fix(ci): use node-oauth/express-oauth-server for integration test
a47e867
@jankapunkt
fix(ci): fix path for integration test
8073105
@jankapunkt
fix(ci): fix path for integration test
06e6b4c
@jankapunkt
fix(ci): fix path for integration test
099d063
@jankapunkt
fix(ci): exclude node 16 to get running
f4bfae8
@jankapunkt
build(ci): add --access-public flag to npm publish
9c70236
@jankapunkt
ci: disable release workflow until decision is made pro automated rel…
c97d0e3 
…eases
@jankapunkt
fix(core): Bearer regular expression matching in authenticate handler #…
72538a3 
…105

Merge pull request #105 from FStefanni/89_15_614
@jankapunkt
feature(ci): release workflow added #117
f4caeb6
@jankapunkt
refactor(ci): remove unused ci workflow
08d4cd3
@jankapunkt jankapunkt closed this Mar 18, 2022
@jankapunkt jankapunkt deleted the remove-release-workflow branch March 18, 2022 11:08
@jankapunkt jankapunkt restored the remove-release-workflow branch March 18, 2022 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@jankapunkt @HappyZombies @jwerre @oklas @jorenvandeweyer @orgads @FStefanni